Skip to content

Patch empty history printing#27

Merged
kubasaw merged 2 commits intomasterfrom
Patch-empty-history-printer-on-master-branch
Feb 8, 2025
Merged

Patch empty history printing#27
kubasaw merged 2 commits intomasterfrom
Patch-empty-history-printer-on-master-branch

Conversation

@kubasaw
Copy link
Member

@kubasaw kubasaw commented Feb 8, 2025

No description provided.

Copy link

@ramundomario ramundomario left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, was just getting this now with v0.14.1-beta.0:

aiohttp.client_exceptions.ClientConnectorCertificateError: Cannot connect to host account2.hon-smarthome.com:443 ssl:True [SSLCertVerificationError: (1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1018)')]

During handling of the above exception, another exception occurred:

Traceback (most recent call last):

  ...

  File "/usr/local/lib/python3.13/site-packages/pyhon/apis/wrappers/_base.py", line 113, in _log_history

    self._history[-1].response_text,

    ~~~~~~~~~~~~~^^^^

IndexError: deque index out of range

@kubasaw kubasaw merged commit 4548a65 into master Feb 8, 2025
3 checks passed
@kubasaw kubasaw deleted the Patch-empty-history-printer-on-master-branch branch February 8, 2025 21:06
@kubasaw
Copy link
Member Author

kubasaw commented Feb 9, 2025

@ramundomario Thanks for checking. Code review would make my writing much easier, but there's not always a reviewer ;)

@ramundomario
Copy link

@ramundomario Thanks for checking. Code review would make my writing much easier, but there's not always a reviewer ;)

You're welcome! 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants